From: Jonas Dreßler <verdre@v0yd.nl>
Date: Thu, 11 Mar 2021 14:26:14 +0000 (+0100)
Subject: overviewControls: Return zero preferred size
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~616^2^2~118
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=ae23ad372dde963b6101fd4fa547ba1c15a7fd97;p=gnome-shell.git

overviewControls: Return zero preferred size

The OverviewControls actor gets allocated a fixed size by its parent,
the OverviewActor, anyway, so it's pretty useless to go through the size
request machinery and add up all the sizes of items in the iconGrid,
coming up with a preferred size that's wrong anyway.

Instead simply return a min and preferred size of 0 in
get_preferred_height/width of ControlsManagerLayout.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1755>
---

diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js
index 2a0dd3c66..b84881137 100644
--- a/js/ui/overviewControls.js
+++ b/js/ui/overviewControls.js
@@ -115,6 +115,16 @@ class ControlsManagerLayout extends Clutter.BoxLayout {
         this.hookup_style(container);
     }
 
+    vfunc_get_preferred_width(_container, _forHeight) {
+        // The MonitorConstraint will allocate us a fixed size anyway
+        return [0, 0];
+    }
+
+    vfunc_get_preferred_height(_container, _forWidth) {
+        // The MonitorConstraint will allocate us a fixed size anyway
+        return [0, 0];
+    }
+
     vfunc_allocate(container, box) {
         const childBox = new Clutter.ActorBox();